Skip to content

Conversation

fforjan
Copy link
Contributor

@fforjan fforjan commented Jun 4, 2024

This enables to run unit test in .Net 472.

It also fixes one issue when using .Net472 - we found it when following the steps from the Readme.MD

@fforjan
Copy link
Contributor Author

fforjan commented Jun 4, 2024

Notice that build on linux only run for net80
image

While for windows, you would see:

Failed!  - Failed:    29, Passed:   160, Skipped:     2, Total:   191, Duration: 734 ms - Wasmtime.Tests.dll (net472)
Failed!  - Failed:    19, Passed:    80, Skipped:     0, Total:    99, Duration: 278 ms - Wasmtime.Tests.dll (net8.0)

@jsturtevant
Copy link
Contributor

@fforjan could you rebase this PR? I will take a closer look afterwards

@fforjan fforjan force-pushed the feature/enableNet472OnUnitTests branch from 38cd8bc to 60724e1 Compare July 1, 2024 13:47
@fforjan
Copy link
Contributor Author

fforjan commented Jul 2, 2024

@jsturtevant should be good now

@fforjan
Copy link
Contributor Author

fforjan commented Jul 8, 2024

@jsturtevant let me know if you would like me to do anything

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants